Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support multiple traffic update files, and include filename in debug tile metadata #2098

Merged
merged 1 commit into from
Mar 17, 2016

Conversation

danpat
Copy link
Member

@danpat danpat commented Mar 15, 2016

This change adds support for --segment-speed-file file1 --segment-speed-file file2 to osrm-contract.

It also saves the filenames, and remembers which file was the source of the data used to update each edge weight.

If the --segment-speed-file option is not used, there is no overhead to this feature. When it is used, there is an extra 0.25 * (size of compressed geometries).

Two new files are written out when this feature is used:

.osrm.datasource_indexes <- these link the compressed geometry segments to datasource name that sourced their data
.osrm.datasource_names <- this is the list of datasource names used, in order of priority (least to most)

@TheMarex TheMarex force-pushed the tile_datasource_info branch from 0bfc206 to e8c737d Compare March 15, 2016 20:25
@TheMarex
Copy link
Member

⚠️ rebased on rewrite/new-api.

@danpat danpat changed the title [not ready] support multiple traffic update files, and include filename in debug tile metadata Support multiple traffic update files, and include filename in debug tile metadata Mar 15, 2016
@danpat danpat force-pushed the tile_datasource_info branch from b8b037b to e83bdfb Compare March 15, 2016 23:44
@danpat danpat changed the title Support multiple traffic update files, and include filename in debug tile metadata [not ready] Support multiple traffic update files, and include filename in debug tile metadata Mar 15, 2016
@danpat danpat changed the title [not ready] Support multiple traffic update files, and include filename in debug tile metadata Support multiple traffic update files, and include filename in debug tile metadata Mar 16, 2016
@danpat danpat force-pushed the tile_datasource_info branch from fb95a3c to c026b3a Compare March 16, 2016 00:39
return "UNKNOWN";
}

std::copy(m_datasource_name_data.begin() + m_datasource_name_offsets[datasource_name_id],
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apply clang-format.

@danpat danpat force-pushed the tile_datasource_info branch 4 times, most recently from f934b2a to 71529cb Compare March 17, 2016 18:57
command line, and exposes the file name used for each edge in the debug
tiles.
@danpat danpat force-pushed the tile_datasource_info branch from 71529cb to c96dfef Compare March 17, 2016 19:52
danpat added a commit that referenced this pull request Mar 17, 2016
Support multiple traffic update files, and include filename in debug tile metadata
@danpat danpat merged commit 14fd49a into rewrite/new-api Mar 17, 2016
@danpat danpat deleted the tile_datasource_info branch March 17, 2016 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants